Skip to content

[WIP] feat: Add missing sendEvent #1330

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

javier-aliaga
Copy link
Contributor

@javier-aliaga javier-aliaga commented May 5, 2025

Description

This PR introduces a new feature already present in the durabletask-java library to support raiseing an event directly to another Workflow.

This PR contains:

  • New methods on interface WorkflowContex and implementation on DefaultWorkflowContext
  • E2E tests for the new feature
    • tests are not working as it seems the runtime does not supports this feature as state on this issue

imagen

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #1156

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

Durabletask-java library supports the feature for a Workflow to raise an event directly to another Workflow

Signed-off-by: Javier Aliaga <[email protected]>
@salaboy
Copy link
Collaborator

salaboy commented May 5, 2025

@javier-aliaga great stuff! let me know how this goes I am happy to review when it is out of Draft

@artur-ciocanu
Copy link
Contributor

@javier-aliaga could you please let me know if you intend to complete this PR. Thank you.

@cicoyle
Copy link
Contributor

cicoyle commented Jun 24, 2025

I think we can close this PR out for now. It was determined that this is not exposed upstream, so we cant expose the functionality yet. Once upstream supports it we can proceed with the implementation. Thank you for starting this process & for your efforts here! 🙏🏻

@cicoyle cicoyle closed this Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Workflows] Missing sendEvent feature
4 participants